gpuid-multiple-ids.php: Added option to use multiple fields for count
#1142
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
WalkthroughAdded an elseif branch to GPUID_Multiple_IDs::get_count to accept a Changes
Sequence Diagram(s)sequenceDiagram
actor Caller
participant GPUID as GPUID_Multiple_IDs
participant Entry as EntryStore
Caller->>GPUID: get_count(entry, countArg)
alt countArg contains field_ids array
loop for each field_id
GPUID->>Entry: get value for field_id
Entry-->>GPUID: valueN
end
GPUID->>GPUID: cast to int, abs, sum values
GPUID-->>Caller: return totalSum
else other count formats
GPUID->>GPUID: existing branches (single field, static, etc.)
GPUID-->>Caller: return result
end
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Suggested reviewers
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
✨ Finishing Touches
🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
CodeRabbit Configuration File (
|
codepuncher
force-pushed
the
patch-1
branch
5 times, most recently
from
e4900ba to
04c2030
Compare
…count This is to extend the raffle/lottery ticket example on here: https://gravitywiz.com/gravity-forms-build-raffle-or-lottery/ While the guide supports "packages" of pre-defined counts for choice selections, it does not allow user-defined counts for multiple ticket types. This PR allows you to specify multiple field IDs that will all be counted and totalled to the final count allowing you to sell multiple ticket types with as many as a user would like to purchase.
veryspry
approved these changes
Aug 13, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Context
https://gravitywiz.com/gravity-forms-build-raffle-or-lottery/
Summary
This is to extend the raffle/lottery ticket example on here: https://gravitywiz.com/gravity-forms-build-raffle-or-lottery/
While the guide supports "packages" of pre-defined counts for choice selections, it does not allow user-defined counts for multiple ticket types.
This PR allows you to specify multiple field IDs that will all be counted and totalled to the final count allowing you to sell multiple ticket types with as many as a user would like to purchase.